Skip to content
This repository has been archived by the owner on Aug 1, 2022. It is now read-only.

Network diagnostics #1758

Merged
merged 2 commits into from
May 25, 2021
Merged

Network diagnostics #1758

merged 2 commits into from
May 25, 2021

Conversation

alexjg
Copy link
Contributor

@alexjg alexjg commented Apr 8, 2021

Adds a network diagnostics screen - accessible via a <ctrl>-i shortcut - which shows the waiting room state and any events that change the state of the waiting room.

@alexjg alexjg force-pushed the network-diagnostics branch 2 times, most recently from 832c296 to cb60d28 Compare April 10, 2021 15:13
@FintanH FintanH force-pushed the daemon branch 2 times, most recently from c6dd9d7 to dc720af Compare April 15, 2021 07:33
@FintanH FintanH force-pushed the daemon branch 3 times, most recently from f0e022e to ec4b81a Compare April 30, 2021 09:31
@FintanH FintanH force-pushed the daemon branch 3 times, most recently from 19a36f1 to 3d076fb Compare May 5, 2021 08:05
Base automatically changed from daemon to master May 5, 2021 10:10
@alexjg alexjg marked this pull request as ready for review May 13, 2021 16:00
@alexjg alexjg requested a review from a team as a code owner May 13, 2021 16:00
@alexjg alexjg requested review from a team May 13, 2021 16:00
@alexjg alexjg changed the title WIP: Network diagnostics Network diagnostics May 13, 2021
Copy link
Contributor

@geigerzaehler geigerzaehler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! This should be super helpful 🔍

ui/Screen/NetworkDiagnostics/ConnectedPeers.svelte Outdated Show resolved Hide resolved
ui/Screen/NetworkDiagnostics.svelte Outdated Show resolved Hide resolved
webpack.config.ts Show resolved Hide resolved
ui/src/waitingRoom.ts Outdated Show resolved Hide resolved
ui/src/path.ts Outdated Show resolved Hide resolved
ui/src/screen/network.ts Outdated Show resolved Hide resolved
ui/Screen/NetworkDiagnostics/WaitingRoom.svelte Outdated Show resolved Hide resolved
ui/Screen/NetworkDiagnostics.svelte Outdated Show resolved Hide resolved
@alexjg alexjg mentioned this pull request May 19, 2021
@alexjg alexjg force-pushed the network-diagnostics branch 4 times, most recently from eb30877 to ad34082 Compare May 21, 2021 12:20
Signed-off-by: Alex Good <alex@memoryandthought.me>
@alexjg
Copy link
Contributor Author

alexjg commented May 21, 2021

This should be ready to take another look at. The only remaining irritation is that svelte-json-tree causes a bunch of warnings when bundling. This will be fixed when tanhauhau/svelte-json-tree#10 lands.

@alexjg alexjg requested a review from geigerzaehler May 24, 2021 15:06
Copy link
Contributor

@geigerzaehler geigerzaehler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! 🎆

@juliendonck
Copy link
Member

@brandonhaslegs and I might take a stab at styling this a bit. Should we push to this PR or branch of of this?

@alexjg alexjg merged commit c361e27 into master May 25, 2021
@alexjg alexjg deleted the network-diagnostics branch May 25, 2021 09:35
@geigerzaehler
Copy link
Contributor

@brandonhaslegs and I might take a stab at styling this a bit. Should we push to this PR or branch of of this?

As discussed, we’ll move forward merging this PR and you can create a separate PR to update the styles.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants